With this patch an undefined mask in a shufflevector will be printed as poison.
This change is done to support the new shufflevector semantics for undefined mask elements (see D148637).
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/include/llvm-c/Core.h | ||
---|---|---|
4088 ↗ | (On Diff #516957) | undefined to poison |
Comment Actions
Could you please split the change to printing + the test updates from all the other changes? The code changes get lost in the large diff.
llvm/include/llvm-c/Core.h | ||
---|---|---|
4090 ↗ | (On Diff #516957) | This breaks C API compatibility without good cause. Either leave this name alone or add a function with the new name and mark this one as deprecated. |
Comment Actions
Yes, that makes sense. The internal representation can be addressed separately from the printing.