Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: Quuxplusone/llvm-project
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: Quuxplusone/llvm-project
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: p2752
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 1 commit
  • 5 files changed
  • 1 contributor

Commits on Jan 9, 2023

  1. [clang] Implement P2752R0 "Static storage for braced initializers" 

    TODO FIXME BUG HACK: The last two examples in "static-init-list-regressions.cpp",
involving `decltype(E{1,2,3})`, are noisily broken by this patch. They should
continue to compile okay, but instead the compiler thinks they're invalid.
The workaround is to disable the feature using `-fno-static-init-lists`.
I need to fix this somehow before this would be possible to upstream.

TODO: At `-O1`, LLVM will initialize the stack array from a static global array
named `l_constinit.42` instead of our `_ZL6__il42`. It would be nice if we could
just match that naming convention, but I don't see how. The LLVM codepath is in
`AggExprEmitter::EmitArrayInit()`.
    @Quuxplusone
    Quuxplusone committed Jan 9, 2023
    Configuration menu
    Copy the full SHA
    121a3c4 View commit details
    Browse the repository at this point in the history
Loading