[PATCH] D88457: [scudo][standalone] Fix some primary tests

Roland McGrath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 11:17:33 PDT 2020


mcgrathr added a comment.

Two possible approaches to make this kind of problem less likely to come up again:

1. don't default the template args.  might be annoying for this template.
2. replace bool with an `enum class NamedForPurpose bool : { kThis = false, kThat = true };` in the template argument type


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88457/new/

https://reviews.llvm.org/D88457



More information about the llvm-commits mailing list