[PATCH] D82295: [IR] Short-circuit comparison with itself for Attributes
Danila Malyutin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 23 04:45:10 PDT 2020
danilaml added a comment.
@dblaikie It was actually failing on some of the lit tests when compiler was compiled with EXPENSIVE_CHECKS enabled (kind of how I discovered it), but after this recent refactor which removed a sort step: https://github.com/llvm/llvm-project/commit/ed766f1bb1040a520fb5646ab75851e2b0fd66df#diff-7b43e801c5abbb03d571f9aad6af63a5L792
I don't know if the codepath with ::sort (which has those checks) can be hit from llvm IR. Instead, I've added an explicit check to AttributesTest.cpp that currently asserts.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82295/new/
https://reviews.llvm.org/D82295
More information about the llvm-commits
mailing list