[PATCH] D79885: [FileCheck] Add multiplication support to numerical expressions.

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 13 15:16:53 PDT 2020


thopre added a comment.

In D79885#2034705 <https://reviews.llvm.org/D79885#2034705>, @paulwalker-arm wrote:

> I understand the operator precedence might surprise some (although personally I just assumed a left->right evaluation order) but when taken with named operators (https://reviews.llvm.org/D79882) is it worth enforcing and documenting this simple precedence order rather than complicating things?  With the bracket feature providing a route to a custom evaluation order.


Using the functional notation as I suggested would allow to clarify how is the evaluation to be performed since it forces parenthesis around the arguments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79885





More information about the llvm-commits mailing list